Skip to content

BER-71: Rewrite GymDetailViewController in SwiftUIFeature - #363

Open
YHC66 wants to merge 9 commits into
masterfrom
feature-ber-71
Open

BER-71: Rewrite GymDetailViewController in SwiftUIFeature #363
YHC66 wants to merge 9 commits into
masterfrom
feature-ber-71

Conversation

@YHC66

@YHC66 YHC66 commented Apr 18, 2025

Copy link
Copy Markdown
Contributor

https://linear.app/moon-stone/issue/BER-71/rewrite-gymdetailviewcontroller-in-swiftui

  • Rewrote GymdetailView into SwiftUI

  • Tested and ensured the compatibility of dark mode

Screen.Recording.2025-04-18.at.4.31.52.PM.mov

@YHC66
YHC66 requested review from Copilot and justinwongeecs April 18, 2025 23:36

This comment was marked as off-topic.

@YHC66
YHC66 requested a review from Copilot April 18, 2025 23:47

This comment was marked as duplicate.

@YHC66 YHC66 self-assigned this Apr 18, 2025
@baeuke
baeuke self-requested a review April 24, 2025 00:02
Comment thread berkeley-mobile/Fitness/GymDetailSwiftUIView.swift Outdated
Comment thread berkeley-mobile/Fitness/GymDetailSwiftUIView.swift Outdated
Comment thread berkeley-mobile/Fitness/GymDetailSwiftUIView.swift Outdated
Comment thread berkeley-mobile/Fitness/GymDetailSwiftUIView.swift Outdated
Comment thread berkeley-mobile/Fitness/GymDetailSwiftUIView.swift Outdated
Comment thread berkeley-mobile/Fitness/GymDetailSwiftUIView.swift Outdated
Comment thread berkeley-mobile/Fitness/GymDetailViewController.swift Outdated
@YHC66
YHC66 requested a review from justinwongeecs May 1, 2025 00:39

@justinwongeecs justinwongeecs left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Refactor the overview section into a more generic one that can be suitable for DiningHall, Gym, and Library.

@YHC66

YHC66 commented May 1, 2025

Copy link
Copy Markdown
Contributor Author

Refactor the overview section into a more generic one that can be suitable for DiningHall, Gym, and Library.

A little confused about if I solved it correctly.

@YHC66
YHC66 requested a review from justinwongeecs May 1, 2025 01:51
Comment thread berkeley-mobile/Fitness/GymDetailView.swift Outdated
Comment thread berkeley-mobile/Fitness/GymDetailView.swift Outdated
Comment thread berkeley-mobile/Fitness/GymDetailView.swift Outdated
Comment thread berkeley-mobile/Fitness/GymDetailView.swift Outdated
Comment thread berkeley-mobile/Fitness/GymDetailView.swift Outdated
@YHC66
YHC66 requested a review from justinwongeecs May 1, 2025 18:41
Comment thread berkeley-mobile/Fitness/GymDetailView.swift Outdated
Comment thread berkeley-mobile/Fitness/GymDetailView.swift
Comment thread berkeley-mobile/Fitness/GymDetailView.swift Outdated
Comment thread berkeley-mobile/Fitness/GymDetailView.swift Outdated
@YHC66
YHC66 requested review from baeuke and justinwongeecs May 2, 2025 22:59
@baeuke
baeuke removed their request for review May 5, 2025 00:52
@YHC66 YHC66 added the WIP 🚧 label May 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants